auth.js ➔ ??? - Code Metrics - Inspection of "Tested deregister flow" - emilfolino/order_api - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( a5e25e...81f0bb )

by Emil

created 2019-02-28 13:02 UTC

auth.js ➔ ??? A

↳ Parent: v2/route/auth.js

Complexity

Conditions	1
Paths	1

Size

Total Lines	8
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	3
CRAP Score	1

Importance

Changes

Metric	Value
cc	1
eloc	1
nc	1
nop	2
dl	0
loc	8
ccs	3
cts	3
cp	1
crap	1
rs	10
c	0
b	0
f	0

const express = require('express');
const router = express.Router();

const auth = require("../models/auth.js");

router.get('/api_key', (req, res) => {
    let data = {
        message: "",
        email: ""
    };

    res.render("api_key/form", data);
});

router.post('/api_key/confirmation', (req, res) => {
    if (req.body.gdpr && req.body.gdpr == "gdpr") {
        return auth.getNewAPIKey(res, req.body.email);
    }

    let data = {
        message: "Approve the terms and conditions.",
        email: req.body.email
    };

    res.render("api_key/form", data);

});

router.get('/api_key/deregister', (req, res) => {
    let data = {
        message: "",
        email: "",
        apikey: ""
    };

    res.render("api_key/deregister", data);
});

router.post('/api_key/deregister', (req, res) => {
    if (req.body.email && req.body.apikey &&
        req.body.email.length > 0 && req.body.apikey.length > 0) {
        return auth.deregister(res, req.body);
    }

    let data = {
        message: "Both E-mail and API-key is needed to deregister.",
        email: "",
        apikey: ""
    };

    res.render("api_key/deregister", data);

});

router.post('/login', (req, res) => auth.login(res, req.body));
router.post('/register', (req, res) => auth.register(res, req.body));

module.exports = router;


1	1	const express = require('express');
2	1	const router = express.Router();
3
4	1	const auth = require("../models/auth.js");
5
6	1	router.get('/api_key', (req, res) => {
7	1	let data = {
8		message: "",
9		email: ""
10		};
11
12	1	res.render("api_key/form", data);
13		});
14
15	1	router.post('/api_key/confirmation', (req, res) => {
16	11	if (req.body.gdpr && req.body.gdpr == "gdpr") {
17	9	return auth.getNewAPIKey(res, req.body.email);
18		}
19
20	2	let data = {
21		message: "Approve the terms and conditions.",
22		email: req.body.email
23		};
24
25	2	res.render("api_key/form", data);
		0 ignored issues – show Best Practice introduced 2019-02-27 12:56 UTC by Report Bug Copy Issue Report There is no `return` statement in this branch, but you do return something in other branches. Did you maybe miss it? If you do not want to return anything, consider adding `return undefined;` explicitly. Loading history...
26		});
27
28	1	router.get('/api_key/deregister', (req, res) => {
29	1	let data = {
30		message: "",
31		email: "",
32		apikey: ""
33		};
34
35	1	res.render("api_key/deregister", data);
36		});
37
38	1	router.post('/api_key/deregister', (req, res) => {
39	6	if (req.body.email && req.body.apikey &&
40		req.body.email.length > 0 && req.body.apikey.length > 0) {
41	1	return auth.deregister(res, req.body);
42		}
43
44	2	let data = {
45		message: "Both E-mail and API-key is needed to deregister.",
46		email: "",
47		apikey: ""
48		};
49
50	2	res.render("api_key/deregister", data);
		0 ignored issues – show Best Practice introduced 2019-02-28 08:19 UTC by Report Bug Copy Issue Report There is no `return` statement in this branch, but you do return something in other branches. Did you maybe miss it? If you do not want to return anything, consider adding `return undefined;` explicitly. Loading history...
51		});
52
53	6	router.post('/login', (req, res) => auth.login(res, req.body));
54	5	router.post('/register', (req, res) => auth.register(res, req.body));
55
56		module.exports = router;
57

emilfolino / order_api

Push — master ( a5e25e...81f0bb )

auth.js ➔ ??? A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like